Submodule 적용 #54
Closed
Submodule 적용 #54
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 서브모듈을 통해 private repository 로 yml 을 관리할 수 있게 한다.
- 서브모듈을 체크하웃하도록 한다.
Member
|
과정을 상세히 정리해주셔서 이해하는데 큰 도움이 되었습니다! 제가 이해한 바로는 submodule 자체가 여러 레포지토리에서 공통으로 사용할만한 내용들을 작성해두는 용도로(라이브러리처럼) 사용하는게 더 적합하다고 생각이 드는데, 솔리드 커넥션에서는 민감정보 저장용으로만 사용하게 되니까 큰 이점을 느끼기 어려운 것 같긴하네요ㅜ 그래도 매번 공유하는 수고를 줄일 수 있고, 버전관리에도 이점이 있으니 도입하는데 큰 cost가 들지 않는다면 도입하는것도 좋다고 생각합니다. |
Collaborator
Author
좋습니다! 그럼 우선 이 PR 은 남겨두겠습니다 ㅎㅎ |
wibaek
added a commit
that referenced
this pull request
Dec 16, 2024
수동으로 환경변수에 지정한 yml을 불러오는 것이 아닌, 서브모듈 불러오게 수정 #54 가 적용되었다 생각했는데, closed 되어있어 착각했습니다..
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
관련 이슈
작업 내용
작업 과정을 모두 정리해두었습니다.
https://celestial-sheep-6b7.notion.site/b3f5b05149544e72b15ed6f2999bf6f7
특이 사항
머지가 되어야 깃허브 액션이 잘 동작하는지 확인할 수 있기 때문에
머지 후 더 수정해야 할 수도 있습니다😓
리뷰 요구사항 (선택)
서브모듈을 적용하면 할 수록 단점들이 보이더라고요..
서브모듈에 대한 러닝커브
사실 러닝커브가 높진 않은데, 제가 괜히 욕심을 부려 새로운 기능을 구현하시는 새원님께 공부할걸 드린건 아닌지.. 하는 생각이 들었습니다.
서브모듈이 빌드,배포할 때도 잘 적용되는지 보려면 “깃허브 액션을 돌려야 한다.”
그런데 깃허브 액션에 realse 브랜치에 머지가 될 때 돌아가도록 설정이 되어있습니다. 그래서 지금 이걸 바꾸더라도, ‘잘 배포가 되는지’ 확인하려면 realse 브랜치에 main 브랜치를 머지를 한 뒤여야 할 것입니다. 아마 모든 기능을 구현한 이후이겠지요? 이처럼 확인을 하기까지 오래 걸린다는 점이 있습니다.
사실 yml 공유만 열심히 하기만 하면 된다.
사실 yml 파일을 잘 공유하기만 하면 해결되는 문제라는 생각도 드네요.. 세원님이 주신 application-secret 파일을 적용하니 잘 돌아갔습니다.
그래서 장단점을 비교해봤을 때 개인적으로는 단점이 더 크게 느껴졌습니다. 서브모듈을 도입해서 얻을 수 있는 장점은 1) 민감한 정보 숨기기 2) yml 파일 버전관리하기 정도 3) 새로운 기술에 대한 호기심 해소 정도인데, 1번은 우리 레포 자체가 원래 private 이라 큰 이점은 없고요.. 3번은 지금의 우선순위가 아니니 적용했을 때의 수고로움이 장점보다 더 크지 않나 하는 생각이 듭니다😓
우선 제가 구현은 해두긴 했지만 세원님이 장단점을 살펴보시고 merge 하셔도 되고 / 안하셔도 됩니다! 세원님이 결정하시는대로 따르겠습니다.